gdk: Don't use GDK_THREADS_ENTER/LEAVE macros internally
authorMatthias Clasen <mclasen@redhat.com>
Sat, 28 Jul 2012 12:40:04 +0000 (14:40 +0200)
committerMatthias Clasen <mclasen@redhat.com>
Mon, 30 Jul 2012 16:01:47 +0000 (18:01 +0200)
These are just wrappers for the functions, and we want to
deprecate them. Stopping to use them internally is a good
first step.

gdk/broadway/gdkeventsource.c
gdk/gdk.c
gdk/quartz/gdkeventloop-quartz.c
gdk/quartz/gdkevents-quartz.c
gdk/wayland/gdkeventsource.c
gdk/win32/gdkevents-win32.c
gdk/x11/gdkdisplay-x11.c
gdk/x11/gdkeventsource.c

index 49a88823e4df9519b54ed78d0d816f0693af6dba..14fa4793c0ed74d9822e5b289c610dbb7ad32175 100644 (file)
@@ -58,12 +58,12 @@ gdk_event_source_prepare (GSource *source,
   GdkDisplay *display = ((GdkEventSource*) source)->display;
   gboolean retval;
 
-  GDK_THREADS_ENTER ();
+  gdk_threads_enter ();
 
   *timeout = -1;
   retval = (_gdk_event_queue_find_first (display) != NULL);
 
-  GDK_THREADS_LEAVE ();
+  gdk_threads_leave ();
 
   return retval;
 }
@@ -74,14 +74,14 @@ gdk_event_source_check (GSource *source)
   GdkEventSource *event_source = (GdkEventSource*) source;
   gboolean retval;
 
-  GDK_THREADS_ENTER ();
+  gdk_threads_enter ();
 
   if (event_source->event_poll_fd.revents & G_IO_IN)
     retval = (_gdk_event_queue_find_first (event_source->display) != NULL);
   else
     retval = FALSE;
 
-  GDK_THREADS_LEAVE ();
+  gdk_threads_leave ();
 
   return retval;
 }
@@ -353,7 +353,7 @@ gdk_event_source_dispatch (GSource     *source,
   GdkDisplay *display = ((GdkEventSource*) source)->display;
   GdkEvent *event;
 
-  GDK_THREADS_ENTER ();
+  gdk_threads_enter ();
 
   event = gdk_display_get_event (display);
 
@@ -364,7 +364,7 @@ gdk_event_source_dispatch (GSource     *source,
       gdk_event_free (event);
     }
 
-  GDK_THREADS_LEAVE ();
+  gdk_threads_leave ();
 
   return TRUE;
 }
index d1688a8ee3d9dbf113759d51972ab60159552750..3c8c71153b485a12f3bf755effc1f66d9ff8722b 100644 (file)
--- a/gdk/gdk.c
+++ b/gdk/gdk.c
@@ -24,6 +24,8 @@
 
 #include "config.h"
 
+#define GDK_DISABLE_DEPRECATION_WARNINGS 1
+
 #include "gdkmain.h"
 
 #include "gdkinternals.h"
@@ -757,12 +759,12 @@ gdk_threads_dispatch (gpointer data)
   GdkThreadsDispatch *dispatch = data;
   gboolean ret = FALSE;
 
-  GDK_THREADS_ENTER ();
+  gdk_threads_enter ();
 
   if (!g_source_is_destroyed (g_main_current_source ()))
     ret = dispatch->func (dispatch->data);
 
-  GDK_THREADS_LEAVE ();
+  gdk_threads_leave ();
 
   return ret;
 }
index 4181a5de2498b247508c28433fcd1790f361f3f3..4045234eeab96b970c56cdc95f0a43df23025b83 100644 (file)
@@ -616,14 +616,14 @@ gdk_event_prepare (GSource *source,
 {
   gboolean retval;
 
-  GDK_THREADS_ENTER ();
+  gdk_threads_enter ();
   
   *timeout = -1;
 
   retval = (_gdk_event_queue_find_first (_gdk_display) != NULL ||
            _gdk_quartz_event_loop_check_pending ());
 
-  GDK_THREADS_LEAVE ();
+  gdk_threads_leave ();
 
   return retval;
 }
@@ -633,7 +633,7 @@ gdk_event_check (GSource *source)
 {
   gboolean retval;
 
-  GDK_THREADS_ENTER ();
+  gdk_threads_enter ();
 
   /* Refresh the autorelease pool if we're at the base CFRunLoop level
    * (indicated by current_loop_level) and the base g_main_loop level
@@ -653,7 +653,7 @@ gdk_event_check (GSource *source)
   retval = (_gdk_event_queue_find_first (_gdk_display) != NULL ||
            _gdk_quartz_event_loop_check_pending ());
 
-  GDK_THREADS_LEAVE ();
+  gdk_threads_leave ();
 
   return retval;
 }
@@ -665,7 +665,7 @@ gdk_event_dispatch (GSource     *source,
 {
   GdkEvent *event;
 
-  GDK_THREADS_ENTER ();
+  gdk_threads_enter ();
 
   _gdk_quartz_display_queue_events (_gdk_display);
 
@@ -678,7 +678,7 @@ gdk_event_dispatch (GSource     *source,
       gdk_event_free (event);
     }
 
-  GDK_THREADS_LEAVE ();
+  gdk_threads_leave ();
 
   return TRUE;
 }
index ac60e3361e61776d2676ebd61ab8f720d6510835..4921fcd510aad1809530ce55ad0c9ad28ad5a6f9 100644 (file)
@@ -1478,9 +1478,9 @@ _gdk_quartz_display_queue_events (GdkDisplay *display)
          g_list_free_1 (node);
          gdk_event_free (event);
 
-          GDK_THREADS_LEAVE ();
+          gdk_threads_leave ();
           [NSApp sendEvent:nsevent];
-          GDK_THREADS_ENTER ();
+          gdk_threads_enter ();
         }
 
       _gdk_quartz_event_loop_release_event (nsevent);
index badbaaa2a0fbac324b611ac4c434bc057283f5ef..f86a5802951ffb16655a5d571fbf7178df361805 100644 (file)
@@ -68,7 +68,7 @@ gdk_event_source_dispatch(GSource *base,
   GdkDisplay *display = source->display;
   GdkEvent *event;
 
-  GDK_THREADS_ENTER ();
+  gdk_threads_enter ();
 
   event = gdk_display_get_event (display);
 
@@ -79,7 +79,7 @@ gdk_event_source_dispatch(GSource *base,
       gdk_event_free (event);
     }
 
-  GDK_THREADS_LEAVE ();
+  gdk_threads_leave ();
 
   return TRUE;
 }
index 2bc7c124776ed6715cdedcad6e9397b5fda979bd..962daf2d09e1fc7e4dfb3baa1c2aad10224efed0 100644 (file)
@@ -3326,7 +3326,7 @@ gdk_event_prepare (GSource *source,
 {
   gboolean retval;
 
-  GDK_THREADS_ENTER ();
+  gdk_threads_enter ();
 
   *timeout = -1;
 
@@ -3334,7 +3334,7 @@ gdk_event_prepare (GSource *source,
            (modal_win32_dialog == NULL &&
             GetQueueStatus (QS_ALLINPUT) != 0));
 
-  GDK_THREADS_LEAVE ();
+  gdk_threads_leave ();
 
   return retval;
 }
@@ -3344,7 +3344,7 @@ gdk_event_check (GSource *source)
 {
   gboolean retval;
   
-  GDK_THREADS_ENTER ();
+  gdk_threads_enter ();
 
   if (event_poll_fd.revents & G_IO_IN)
     {
@@ -3357,7 +3357,7 @@ gdk_event_check (GSource *source)
       retval = FALSE;
     }
 
-  GDK_THREADS_LEAVE ();
+  gdk_threads_leave ();
 
   return retval;
 }
@@ -3369,7 +3369,7 @@ gdk_event_dispatch (GSource     *source,
 {
   GdkEvent *event;
  
-  GDK_THREADS_ENTER ();
+  gdk_threads_enter ();
 
   _gdk_win32_display_queue_events (_gdk_display);
   event = _gdk_event_unqueue (_gdk_display);
@@ -3389,7 +3389,7 @@ gdk_event_dispatch (GSource     *source,
        }
     }
   
-  GDK_THREADS_LEAVE ();
+  gdk_threads_leave ();
 
   return TRUE;
 }
index 3ef8fa056741fe34a6ce9e0b16edb994485122fa..0f8b35fc32de0d67e525a6a51166930149653d0e 100644 (file)
@@ -1483,11 +1483,11 @@ process_internal_connection (GIOChannel  *gioc,
 {
   GdkInternalConnection *connection = (GdkInternalConnection *)data;
 
-  GDK_THREADS_ENTER ();
+  gdk_threads_enter ();
 
   XProcessInternalConnection ((Display*)connection->display, connection->fd);
 
-  GDK_THREADS_LEAVE ();
+  gdk_threads_leave ();
 
   return TRUE;
 }
index 615ff9a1d08f6f5e9c790be634906f0eae1a3940..6b82611bfccbb539e9b3e304c4d1b74590e4a07d 100644 (file)
@@ -273,13 +273,13 @@ gdk_event_source_prepare (GSource *source,
   GdkDisplay *display = ((GdkEventSource*) source)->display;
   gboolean retval;
 
-  GDK_THREADS_ENTER ();
+  gdk_threads_enter ();
 
   *timeout = -1;
   retval = (_gdk_event_queue_find_first (display) != NULL ||
             gdk_check_xpending (display));
 
-  GDK_THREADS_LEAVE ();
+  gdk_threads_leave ();
 
   return retval;
 }
@@ -290,7 +290,7 @@ gdk_event_source_check (GSource *source)
   GdkEventSource *event_source = (GdkEventSource*) source;
   gboolean retval;
 
-  GDK_THREADS_ENTER ();
+  gdk_threads_enter ();
 
   if (event_source->event_poll_fd.revents & G_IO_IN)
     retval = (_gdk_event_queue_find_first (event_source->display) != NULL ||
@@ -298,7 +298,7 @@ gdk_event_source_check (GSource *source)
   else
     retval = FALSE;
 
-  GDK_THREADS_LEAVE ();
+  gdk_threads_leave ();
 
   return retval;
 }
@@ -349,7 +349,7 @@ gdk_event_source_dispatch (GSource     *source,
   GdkDisplay *display = ((GdkEventSource*) source)->display;
   GdkEvent *event;
 
-  GDK_THREADS_ENTER ();
+  gdk_threads_enter ();
 
   event = gdk_display_get_event (display);
 
@@ -360,7 +360,7 @@ gdk_event_source_dispatch (GSource     *source,
       gdk_event_free (event);
     }
 
-  GDK_THREADS_LEAVE ();
+  gdk_threads_leave ();
 
   return TRUE;
 }